fix: Issue cleanup#92
Merged
Merged
Conversation
✅ Deploy Preview for calypr ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Signed-off-by: Liam Beckman <lbeckman314@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Overview 🌀
This PR replaces the custom
scripts/prepare_docs.pystaging approach withmkdocs-multirepo-plugin, and resolves all Zensical build warnings across imported and legacy docs.Current Behavior⚠️
Upstream tool docs (git-drs, syfon) are synced into
.generated/docs/viascripts/prepare_docs.pybefore each build. This staging step is required locally and in CI, and Zensical reports 22 warnings from bracketed literals in legacy docs content.New Behavior ✔️
mkdocs-multirepo-pluginfetches upstream tool docs from GitHub automatically at build time. Zensical reads directly fromdocs/with no staging step, and builds cleanly with 0 warnings.Build pipeline
scripts/prepare_docs.pyand themake prepare/make synctargetsmkdocs-multirepo-plugintorequirements.txtand configured it inzensical.tomlunder[plugins.multirepo]Docs content fixes (22 → 0 warnings)
[]string,map[string]string, etc.) in backticks in sifter docs[SQLite]missing URL and duplicate[psql]definition in grip databases doc[SIGNED]toSIGNEDin calypr-admin docs